Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Boards removal #346

Merged
merged 5 commits into from
Aug 17, 2023
Merged

Boards removal #346

merged 5 commits into from
Aug 17, 2023

Conversation

esethna
Copy link
Contributor

@esethna esethna commented Aug 16, 2023

Add notice about boards removal in Cloud

@amyblais can you help coordinate getting this QA'd and merged asap?

It's important to test that

  1. This only appears on Cloud instances (not on-prem)
  2. This only appears for sys admins

@amyblais amyblais added the 2: QA Review Requires review by a QA tester label Aug 16, 2023
@amyblais
Copy link
Member

@lindy65 @lindalumitchell @yasserfaraazkhan I've added you all as reviewers as I wasn't sure who is available from QA to test this today.

amyblais
amyblais previously approved these changes Aug 16, 2023
@amyblais amyblais self-assigned this Aug 16, 2023
@amyblais amyblais dismissed stale reviews from yasserfaraazkhan and themself via 607b206 August 16, 2023 17:04
@amyblais
Copy link
Member

/update-branch

@amyblais
Copy link
Member

I opened a ticket for checking why the build is failing here https://mattermost.atlassian.net/browse/CLD-6146.

notices.json Outdated Show resolved Hide resolved
Co-authored-by: Akis Maziotis <[email protected]>
Copy link
Contributor

@yasserfaraazkhan yasserfaraazkhan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified that it is only to cloud license instances and only to system admin.
(test instance)

Screenshot 2023-08-17 at 5 42 41 PM

@amyblais amyblais added 3: Reviews Complete All reviewers have approved the pull request and removed 2: QA Review Requires review by a QA tester labels Aug 17, 2023
@amyblais amyblais merged commit 516e829 into release Aug 17, 2023
1 check passed
@amyblais amyblais deleted the boards_removal branch August 17, 2023 12:57
@amyblais amyblais mentioned this pull request Aug 25, 2023
1 task
"localizedMessages": {
"en": {
"title": "Mattermost Boards will be removed on September 28, 2023",
"description": "Mattermost Boards is transitioning to a fully community supported plugin. As a result, Boards will be removed from Mattermost Cloud instances on September 21, 2023. Click below to learn more about how this impacts your instance and how to prepare.",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@amyblais looks like this date was missed in the description when we sent the notice, can you help update it in a new PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants